Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add StatsD support #24

Merged
merged 6 commits into from
Dec 19, 2016
Merged

Add StatsD support #24

merged 6 commits into from
Dec 19, 2016

Conversation

callumj
Copy link
Contributor

@callumj callumj commented Dec 19, 2016

This PR adds support for switching the reporting backend to StatsD by specifying -backend statsd. When specified this will use the StatsD backend (default to CW) and report similar metrics.

This implementation uses the Datadog StatsD client which supports tagging when sent to a compatible server like the Datadog agent. To support regular StatsD implementations I've added -statsd-tags move the segmentation by pipeline and queue to a tag.

@lox
Copy link
Contributor

lox commented Dec 19, 2016

Looks great, I glanced over the code and don't have any specific feedback. Happy to get this merged.

Could I trouble you to update the README file to reflect the change and include an example of use?

@callumj
Copy link
Contributor Author

callumj commented Dec 19, 2016

Could I trouble you to update the README file to reflect the change and include an example of use?

Sure thing, I've added some info to the README. Let me know if that looks 🆒

@lox lox merged commit 1e478dc into buildkite:master Dec 19, 2016
@lox
Copy link
Contributor

lox commented Dec 19, 2016

Merged this into master and it will make the next release. For now:

https://buildkite-metrics.s3.amazonaws.com/buildkite-metrics-Linux-x86_64-v1.3.0-7-g1e478dc
https://buildkite-metrics.s3.amazonaws.com/buildkite-metrics-v1.3.0-7-g1e478dc-lambda.zip

Thanks for the contribution @callumj.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants